Skip to content

Cursor test - #617

Closed
cursor[bot] wants to merge 1 commit into
masterfrom
cursor/test-steps-scoped-timeout-guard-c2e8
Closed

cursor[bot] wants to merge 1 commit into
masterfrom
cursor/test-steps-scoped-timeout-guard-c2e8

Conversation

@cursor

@cursor cursor Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

No description provided.

Add comprehensive test cases to test_tcp_socket.cpp that validate the
scoped receive-timeout guard feature described in issue #609. The guard
is an RAII helper on TCPSocket that temporarily raises the receive
timeout and restores it on destruction, replacing manual save/restore
patterns in the dashboard clients and the private helper in rtde_client.

Test cases cover:
- Basic raise and restore on scope exit
- Guard does not lower an already-higher timeout
- Guard raises timeout when current is lower
- Guard is a no-op when no timeout is configured (blocking socket)
- Timeout restored even when scope exits via exception
- Sub-second precision (e.g. 1.5s = 1s + 500000us)
- Nested guards restore in correct order
- Equal timeout request is a no-op
- Zero minimum is a no-op
- Move semantics (moved-from guard does not double-restore)
@urrsk urrsk closed this Nov 4, 2026
@urrsk
urrsk deleted the cursor/test-steps-scoped-timeout-guard-c2e8 branch November 4, 2026 12:28
@urrsk urrsk changed the title Add test steps for scoped receive-timeout guard (issue #609) Cursor test Nov 4, 2026
@UniversalRobots UniversalRobots deleted a comment from cursor Bot Nov 4, 2026
@UniversalRobots UniversalRobots deleted a comment from codecov Bot Nov 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants